Skip to content

Conversation

@DaleSeo
Copy link
Contributor

@DaleSeo DaleSeo commented Sep 8, 2025

This PR introduces the bare minimum for code coverage reporting using cargo-llvm-cov and integrates with Codecov. It adds a new coverage job to the CI workflow that generates and uploads coverage reporting in parallel with existing tests. The setup mirrors that of Router, except it uses nextest instead of the built-in test runner and CircleCI instead of GitHub Actions.

For more details, check out the last successful workflow run here: https://github.com/apollographql/apollo-mcp-server/actions/runs/17557278384/job/49864559944

You can also view the initial coverage generated from this feature branch on Codecov here: https://app.codecov.io/gh/apollographql/apollo-mcp-server

2025-09-08 at 12 27 59

If you want to check coverage locally, run this command in the terminal. It will open the coverage report in your browse

$ cargo llvm-cov --all-features --workspace --open

If you've worked on Router, you should already have the llvm-cov subcommand installed. If not, you can install it as follows:

$ cargo install cargo-llvm-cov
2025-09-08 at 12 36 38

Once this PR is merged into develop and we start collecting coverage there, I'll follow up with additional settings such as thresholds, docs, badges, and more.

@apollo-librarian
Copy link

apollo-librarian bot commented Sep 8, 2025

✅ Docs preview has no changes

The preview was not built because there were no changes.

Build ID: c44abc6e60a3940e749494e8
Build Logs: View logs

@DaleSeo DaleSeo changed the title feat: Implement Test Coverage Measurement and Reporting Implement Test Coverage Measurement and Reporting Sep 8, 2025
@DaleSeo DaleSeo force-pushed the GT-370 branch 3 times, most recently from 2c70034 to e8d46b7 Compare September 8, 2025 15:51
@DaleSeo DaleSeo self-assigned this Sep 8, 2025
@DaleSeo DaleSeo changed the base branch from main to develop September 8, 2025 16:13
@DaleSeo DaleSeo marked this pull request as ready for review September 8, 2025 16:57
@DaleSeo DaleSeo requested a review from a team as a code owner September 8, 2025 16:57
@DaleSeo DaleSeo merged commit f00da39 into develop Sep 8, 2025
9 checks passed
@DaleSeo DaleSeo deleted the GT-370 branch September 8, 2025 19:08
@apollo-bot2 apollo-bot2 mentioned this pull request Sep 12, 2025
@esilverm esilverm mentioned this pull request Sep 12, 2025
@apollo-bot2 apollo-bot2 mentioned this pull request Sep 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants